-
Notifications
You must be signed in to change notification settings - Fork 163
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
最新のWindows SDKでビルドできるようにする #1621
Conversation
✅ Build sakura 1.0.3661 completed (commit 545e3a2aad by @dep5) |
はじめましてっす。
|
ありがとうございます。削りました |
それ、削っていいんですかね。 VS2019のデフォルトインストールされるSDKバージョンが、どこかのタイミングで18362から変更されたので、「SDKバージョンの固定をやめてVSの構成を原因としたビルドエラーの可能性を減らす」という提案だと理解しました。 |
✅ Build sakura 1.0.3662 completed (commit f188088b67 by @dep5) |
エラーを気にせず使っていたのでわかりませんでしたが、 |
README.mdの記述から考えると、「サクラエディタのビルド環境を簡単に構築するための項目」と表現できると思います。 したがって、このファイルからSDKのコンポーネントに関する記述を削れば、SDKがインストールされていなくても通知されなくなるので、後から手動で入れる手間が発生します。 |
This reverts commit e1b12fa.
試しにVisual StudioインストーラーでWindows SDKのチェックを全部外してみましたが |
Kudos, SonarCloud Quality Gate passed! |
✅ Build sakura 1.0.3666 completed (commit 9749c44f95 by @dep5) |
.vsconfigのエラーとはソリューションエクスプローラーの Visual StudioインストーラーでSDKの最新バージョンのみにチェックを入れ直しています。 .vsconfigの更新ありがとうございました。 |
画像のメッセージはエラーではないのでビルドができるのであれば気にしなくても大丈夫だとは思います。 ご確認ありがとうございます。 |
ビルド環境を特定バージョンのwin10SDKに制限する必要は必ずしもないので、変更は適切と思います。 そもそも、ビルドで使える機能はWindows7相当の機能に限定されるようになっていますし、仮にこの設定で最新windowsの機能を利用したPRを投げた場合には、CIでvs2017ビルドが失敗して異変を察知できる仕組みになっています。 特に反対意見等なければ、明日あたりマージしてしまおうと思います。 |
マージしちゃいます。 |
PR の目的
Visual Studio 2019で最新のWindows SDKを使えるようにします
カテゴリ
PR の背景
Visual Studio 2019でのローカルビルドする時Windows SDKの最新バージョンのみをインストールした状態では
指定のバージョンのSDKを入れるようにエラーが出ます
PR のメリット
Windows SDKの最新バージョンが使えます
サクラエディタのビルドでしか使わないバージョンのWindows SDKを入れる必要がなくなります
PR のデメリット (トレードオフとかあれば)
Windows SDKの最新バージョンでは使えない機能があるかもしれません
その時に改めてバージョンを指定してもよいのでは、と思います
仕様・動作説明
Visual Studio 2017では指定のWindows SDKのバージョンを使い
Visual Studio 2019では最新のWindows SDKを使えるようになります
PR の影響範囲
テスト内容
テスト1
手順
関連 issue, PR
参考資料